fasttext-wheel: add build-fasttext-wheel.yml for riscv64 wheels - #1621
Merged
Conversation
luhenry
added a commit
that referenced
this pull request
Sep 8, 2026
luhenry
force-pushed
the
fasttext-wheel
branch
2 times, most recently
from
September 8, 2026 15:06
6fd64be to
d00c113
Compare
Builds fastText's pybind11 extension from facebookresearch/fastText v0.9.2, renamed to the fasttext-wheel PyPI distribution messense publishes it under, with patches fixing a numpy 2.x regression in predict()/scores() and a riscv64-only missing <cstdint> include (backported from an upstream commit that fixed the same failure under GCC 13, before the repo was archived). setup.py has no pyproject.toml declaring pybind11 as a build dependency; its get_pybind_include self-installs pybind11 via `pip install` when missing, but the isolated build venv `python -m build` creates has no pip module, so that self-install fails. Pre-install pybind11/setuptools/numpy (its install_requires) and build without isolation instead.
luhenry
force-pushed
the
fasttext-wheel
branch
from
September 8, 2026 15:40
d00c113 to
cb89cbf
Compare
luhenry
added a commit
that referenced
this pull request
Sep 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
fasttext-wheel0.9.2Compiles fastText's pybind11 C++ extension (word representations and text classification) from the facebookresearch/fastText source that messense's
fasttext-wheelPyPI distribution renames and republishes. Upstream publishes no riscv64 wheel.Mirrors messense/fasttext-wheel's
Linux.yml, rebuilt as a cibuildwheel build-from-checkout against the unmodified facebookresearch/fastText tag instead of the shell-script manylinux loop it uses.Differs from upstream
docker run+auditwheelloop - matches this repo's standard shape, same compiled output.Testing
runtests.py -uunit suite (gen_unit_tests) directly against the installedfasttext.testssubpackage, sinceruntests.pyitself lives outside the wheel.License: OK
Patches
0001-setup-rename-distribution-to-fasttext-wheel.patch- Inappropriate, downstream naming. Renames the PyPI distribution the way messense's own build script does; without it the wheel would be namedfasttext, notfasttext-wheel.0002-FastText-fix-numpy-2.x-copy-False-compatibility.patch- To upstream, blocked (facebookresearch/fastText archived 2024-03-19).np.array(x, copy=False)raises under numpy>=2.0, breakingpredict()/scores(); reproduces off riscv64.Built on cp312; 83 passed, 0 skipped.